builder-tool: Rewrite top/left-attach
authorMatthias Clasen <mclasen@redhat.com>
Sun, 2 Aug 2020 19:10:18 +0000 (15:10 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Sun, 2 Aug 2020 21:58:03 +0000 (17:58 -0400)
Rewrite the GtkGrid:top/left-attach child properties
to the GtkGridLayoutChild:row/column layout properties.

Update expected test output to match.

gtk/tools/gtk-builder-tool-simplify.c
testsuite/tools/simplify-data-3to4/grid.expected
testsuite/tools/simplify-data-3to4/office-runner.expected

index da234e03949af24833165bcee5845d44e1aa578d..92a126ca33ce01da319b2ea823adfbffabc30419 100644 (file)
@@ -1434,21 +1434,21 @@ rewrite_grid_layout_prop (Element *element,
                           const char *old_value,
                           const char *new_value)
 {
-  char *canonical_name;
-
-  canonical_name = g_strdup (old_value);
-  g_strdelimit (canonical_name, "_", '-');
-
   if (g_str_equal (element->element_name, "property"))
     {
+      char *canonical_name;
+
+      canonical_name = g_strdup (old_value);
+      g_strdelimit (canonical_name, "_", '-');
+
       if (has_attribute (element, attr_name, old_value) ||
           has_attribute (element, attr_name, canonical_name))
         {
           set_attribute_value (element, attr_name, new_value);
         }
-    }
 
-  g_free (canonical_name);
+      g_free (canonical_name);
+    }
 }
 
 static void
@@ -1460,6 +1460,8 @@ rewrite_grid_layout (Element *element,
     const char *old_value;
     const char *new_value;
   } props[] = {
+    { "name", "left_attach", "column", },
+    { "name", "top_attach", "row", },
     { "name", "width", "column-span", },
     { "name", "height", "row-span", },
   };
index f14ad0d2489d3a68c2a23fad5ce21c7b7e0e5732..b832c4ba316ae8c00bdbabb60c693f3d8b43bd2e 100644 (file)
@@ -15,8 +15,8 @@
             <property name="margin-top">10</property>
             <property name="margin-bottom">10</property>
             <layout>
-              <property name="left_attach">0</property>
-              <property name="top_attach">0</property>
+              <property name="column">0</property>
+              <property name="row">0</property>
             </layout>
           </object>
         </child>
@@ -25,8 +25,8 @@
             <property name="label" translatable="yes">Hello World!</property>
             <property name="receives_default">1</property>
             <layout>
-              <property name="left_attach">1</property>
-              <property name="top_attach">0</property>
+              <property name="column">1</property>
+              <property name="row">0</property>
             </layout>
           </object>
         </child>
@@ -35,8 +35,8 @@
             <property name="label" translatable="yes">Hello World!</property>
             <property name="receives_default">1</property>
             <layout>
-              <property name="left_attach">2</property>
-              <property name="top_attach">0</property>
+              <property name="column">2</property>
+              <property name="row">0</property>
             </layout>
           </object>
         </child>
@@ -47,8 +47,8 @@
             <property name="group">button3</property>
             <property name="receives_default">1</property>
             <layout>
-              <property name="left_attach">3</property>
-              <property name="top_attach">0</property>
+              <property name="column">3</property>
+              <property name="row">0</property>
             </layout>
           </object>
         </child>
@@ -57,8 +57,8 @@
             <property name="active">1</property>
             <property name="receives_default">1</property>
             <layout>
-              <property name="left_attach">4</property>
-              <property name="top_attach">0</property>
+              <property name="column">4</property>
+              <property name="row">0</property>
             </layout>
           </object>
         </child>
@@ -66,8 +66,8 @@
           <object class="GtkSwitch" id="button6">
             <property name="receives_default">1</property>
             <layout>
-              <property name="left_attach">5</property>
-              <property name="top_attach">0</property>
+              <property name="column">5</property>
+              <property name="row">0</property>
             </layout>
           </object>
         </child>
@@ -76,8 +76,8 @@
             <property name="receives_default">1</property>
             <property name="label">Text Button</property>
             <layout>
-              <property name="left_attach">0</property>
-              <property name="top_attach">1</property>
+              <property name="column">0</property>
+              <property name="row">1</property>
             </layout>
           </object>
         </child>
@@ -87,8 +87,8 @@
             <property name="halign">center</property>
             <property name="valign">center</property>
             <layout>
-              <property name="left_attach">1</property>
-              <property name="top_attach">1</property>
+              <property name="column">1</property>
+              <property name="row">1</property>
             </layout>
           </object>
         </child>
@@ -98,8 +98,8 @@
             <property name="halign">center</property>
             <property name="valign">center</property>
             <layout>
-              <property name="left_attach">2</property>
-              <property name="top_attach">1</property>
+              <property name="column">2</property>
+              <property name="row">1</property>
               <property name="column-span">2</property>
               <property name="row-span">2</property>
             </layout>
index 11131a71e5751bd02a4c2d6970b48f52efea7787..adeca878bf5180e07392772999e122e2eee46cda 100644 (file)
@@ -26,8 +26,8 @@
                       <object class="GtkImage" id="run_image">
                         <property name="vexpand">1</property>
                         <layout>
-                          <property name="left_attach">0</property>
-                          <property name="top_attach">0</property>
+                          <property name="column">0</property>
+                          <property name="row">0</property>
                         </layout>
                       </object>
                     </child>
@@ -41,8 +41,8 @@
 </property>
                         <property name="use_markup">1</property>
                         <layout>
-                          <property name="left_attach">1</property>
-                          <property name="top_attach">1</property>
+                          <property name="column">1</property>
+                          <property name="row">1</property>
                         </layout>
                       </object>
                     </child>
@@ -53,8 +53,8 @@
                           <attribute name="scale" value="2"></attribute>
                         </attributes>
                         <layout>
-                          <property name="left_attach">1</property>
-                          <property name="top_attach">0</property>
+                          <property name="column">1</property>
+                          <property name="row">0</property>
                         </layout>
                       </object>
                     </child>
@@ -81,8 +81,8 @@
                       <object class="GtkImage" id="time_image">
                         <property name="vexpand">1</property>
                         <layout>
-                          <property name="left_attach">1</property>
-                          <property name="top_attach">0</property>
+                          <property name="column">1</property>
+                          <property name="row">0</property>
                         </layout>
                       </object>
                     </child>
@@ -96,8 +96,8 @@
                           <attribute name="font-features" value="tnum 1"></attribute>
                         </attributes>
                         <layout>
-                          <property name="left_attach">0</property>
-                          <property name="top_attach">0</property>
+                          <property name="column">0</property>
+                          <property name="row">0</property>
                         </layout>
                       </object>
                     </child>
                     <child>
                       <object class="GtkImage" id="trophy_image">
                         <layout>
-                          <property name="left_attach">0</property>
-                          <property name="top_attach">0</property>
+                          <property name="column">0</property>
+                          <property name="row">0</property>
                         </layout>
                       </object>
                     </child>
                           </packing>
                         </child>
                         <layout>
-                          <property name="left_attach">1</property>
-                          <property name="top_attach">0</property>
+                          <property name="column">1</property>
+                          <property name="row">0</property>
                         </layout>
                       </object>
                     </child>